Skip to content

Add launcher type and cluster URL to root execution span#255

Open
morgan-wowk wants to merge 1 commit into
execution-tracing-launcher-datafrom
execution-tracing-launcher-type
Open

Add launcher type and cluster URL to root execution span#255
morgan-wowk wants to merge 1 commit into
execution-tracing-launcher-datafrom
execution-tracing-launcher-type

Conversation

@morgan-wowk
Copy link
Copy Markdown
Collaborator

@morgan-wowk morgan-wowk commented May 22, 2026

Add execution.launcher and execution.cloud_provider OTel trace attributes to root execution span

execution.launcher is derived from the top-level key of launcher_data (e.g. kubernetes, kubernetes_job, skypilot) and distinguishes the launcher mechanism used for a given execution.

k8s.cluster.url on the root span allows GKE vs Nebius cluster identification by URL pattern in oasis-backend's multi-cloud setup, populated from cluster_server inside the launcher's data block.

execution.cloud_provider is read from the cloud-pipelines.net/orchestration/cloud_provider task_spec annotation, set at routing time by callers such as MultiLauncherContainerLauncher. This enables traces to be searched by cloud provider (e.g. gke, nebius) without relying on URL pattern matching against cluster hostnames. Launchers with a fixed cloud affinity can also set this annotation directly.

The CLOUD_PROVIDER_ANNOTATION_KEY constant is defined in common_annotations so it can be shared across launchers and the tracing layer without duplication.

Screenshots

Screenshot 2026-05-22 at 8.19.10 PM.png

Copy link
Copy Markdown
Collaborator Author

execution.launcher = top-level key from launcher_data (e.g. 'kubernetes',
'kubernetes_job', 'skypilot') distinguishes launcher mechanism.
k8s.cluster.url on root span allows GKE vs Nebius cluster identification
by URL pattern in oasis-backend's multi-cloud setup.
@morgan-wowk morgan-wowk force-pushed the execution-tracing-launcher-data branch from 3e85d85 to 239a70c Compare May 23, 2026 02:57
@morgan-wowk morgan-wowk force-pushed the execution-tracing-launcher-type branch from 6f8a30a to a8e9b03 Compare May 23, 2026 02:57
@morgan-wowk morgan-wowk marked this pull request as ready for review May 23, 2026 03:22
@morgan-wowk morgan-wowk requested a review from Ark-kun as a code owner May 23, 2026 03:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant